Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

fix(navigation): ignore unknown JS errors in IE #4149

Merged
merged 1 commit into from
Mar 15, 2017
Merged

fix(navigation): ignore unknown JS errors in IE #4149

merged 1 commit into from
Mar 15, 2017

Conversation

thorn0
Copy link
Contributor

@thorn0 thorn0 commented Mar 15, 2017

The err object doesn't have the code property any more (Selenium Server Standalone 3.3.1 + IEDriver win32 3.3.0), so we need a new way to detect those errors. See #841

The `err` object doesn't have the `code` property any more (Selenium Server Standalone 3.3.1 + IEDriver win32 3.3.0), so we need a new way to detect those errors. See #841
@thorn0
Copy link
Contributor Author

thorn0 commented Mar 15, 2017

This might be related to #4143.

Copy link
Member

@cnishina cnishina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@cnishina cnishina merged commit 0eb5b76 into angular:master Mar 15, 2017
@thorn0
Copy link
Contributor Author

thorn0 commented Mar 16, 2017

Can we have 5.1.2 or at least a patch release with just this fix?
In my observation, IE got flakier after upgrading to Protractor 5, and this fix seems to repair it.

@heathkit
Copy link
Contributor

Thanks for the fix! I also need this, I'll ping the caretakers to get a new release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants